Conversation
golangci-lint 2.8.0 is built with Go 1.26, which causes a panic when linting code pinned to Go 1.25 ("file requires newer Go version go1.26"). Downgrade golangci-lint to a release built against Go 1.25 to match the toolchain specified in .tool-versions.
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 6f2f47a. Bugbot is set up for automated code reviews on this repo. Configure here. |
❌ 10 Tests Failed:
View the full list of 10 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
There was a problem hiding this comment.
LGTM, simple version pin to keep the linter compatible with the Go 1.25 toolchain.
Extended reasoning...
Overview
This PR makes a single one-line change to .tool-versions, downgrading golangci-lint from 2.8.0 to 2.7.2 so that it is built against Go 1.25 (matching the repo's pinned golang 1.25.9). Without this, the linter panics with file requires newer Go version go1.26.
Security risks
None. This is a tooling/version-pin change that only affects local development and CI lint runs — no runtime, production, or dependency-graph impact.
Level of scrutiny
Minimal. This is a mechanical config tweak in .tool-versions, the kind of change that's trivially verifiable and reversible. CI will surface any issue with the pinned version immediately.
Other factors
No bugs were found by the bug hunting system. The change matches the description, the rationale is clearly explained, and Cursor Bugbot also flagged it as low risk.
golangci-lint 2.8.0 is built with Go 1.26, which causes a panic when linting code pinned to Go 1.25 ("file requires newer Go version go1.26"). Downgrade golangci-lint to a release built against Go 1.25 to match the toolchain specified in .tool-versions.